-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #5902: correct reporting on ssh technique on aix when service is a... #591
Fixes #5902: correct reporting on ssh technique on aix when service is a... #591
Conversation
"dummy_variable" | ||
string => "dummyvalue", | ||
ifvarclass => "${class_prefix}_activated_on_boot_kept", | ||
classes => always("${class_prefix}_activated_on_boot_kept"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think adding scope => namespace to the original class would be simpler.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait, does this work with 3.5 ??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome, it does work
I thought it was 3.6 specific
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
63bdf8b
to
620f23c
Compare
PR has been updated |
…s already defined at boot
620f23c
to
d490f6f
Compare
OK, merging |
…on_boot Fixes #5902: correct reporting on ssh technique on aix when service is a...
…ckage_manpages_properly Fixes #6092: Package manpages properly
...lready defined at boot